Skip to content

[voice_text] Load db_path, license_path at library binary path by default#559

Merged
k-okada merged 2 commits intojsk-ros-pkg:masterfrom
mqcmd196:voice-text-backward
Oct 20, 2025
Merged

[voice_text] Load db_path, license_path at library binary path by default#559
k-okada merged 2 commits intojsk-ros-pkg:masterfrom
mqcmd196:voice-text-backward

Conversation

@mqcmd196
Copy link
Member

The PR #395 breaks backward compatibility of Hiro. I'm sorry -> @pazeshun @nakagori001 ...

I modified it so that db_path and license_path are not set to their defaults, but instead search for the old paths.

@mqcmd196 mqcmd196 marked this pull request as draft October 20, 2025 04:59
@mqcmd196
Copy link
Member Author

Tested on

  • Hiro (SDK)
  • PR2 (SDK)
  • Spot (API)

@pazeshun I've tested on Hiro by creating temporary workspace. Could you run it from the bringup launch?

@mqcmd196 mqcmd196 marked this pull request as ready for review October 20, 2025 05:25
@mqcmd196 mqcmd196 requested review from k-okada and pazeshun October 20, 2025 05:25
Copy link
Contributor

@pazeshun pazeshun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I confirmed this PR fixes the problem on real Hiro.
Thank you!

@mqcmd196
Copy link
Member Author

Thank you!
@k-okada Could you merge this?

@k-okada k-okada merged commit a3c5e99 into jsk-ros-pkg:master Oct 20, 2025
27 of 28 checks passed
@mqcmd196 mqcmd196 deleted the voice-text-backward branch October 20, 2025 21:25
@mqcmd196
Copy link
Member Author

mqcmd196 commented Oct 24, 2025

@k-okada

Could you add hacktoberfest topic ?

image

@k-okada
Copy link
Member

k-okada commented Oct 24, 2025

done
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants